Skip to content

Migrate github workflow to pyansys/actions. #81

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Dec 1, 2022

Conversation

Revathyvenugopal162
Copy link
Collaborator

@Revathyvenugopal162 Revathyvenugopal162 commented Nov 30, 2022

Fix #80

Overview

PyAnsys/actions is the repository containing collection of reusable workflows and we are making use of that actions instead of replicating it again and again, so that we can reduce the maintenance and lines of codes. I am keeping the test as it is, because it needs docker.

Notes:

Release is skipped to private and public PyPI by commenting.

@Revathyvenugopal162 Revathyvenugopal162 changed the title initial commit for the actions change Migrate github workflow to pyansys/actions. Nov 30, 2022
@github-actions github-actions bot added maintenance Package and maintenance related CI/CD labels Nov 30, 2022
@github-actions github-actions bot added CI/CD and removed CI/CD labels Nov 30, 2022
@github-actions github-actions bot added CI/CD and removed CI/CD labels Nov 30, 2022
@Revathyvenugopal162 Revathyvenugopal162 marked this pull request as ready for review November 30, 2022 16:13
echambla
echambla previously approved these changes Dec 1, 2022
MaxJPRey
MaxJPRey previously approved these changes Dec 1, 2022
Copy link
Contributor

@MaxJPRey MaxJPRey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@Revathyvenugopal162 Revathyvenugopal162 dismissed stale reviews from MaxJPRey and echambla via 1bf06f5 December 1, 2022 09:58
@github-actions github-actions bot added CI/CD and removed CI/CD labels Dec 1, 2022
@github-actions github-actions bot added CI/CD and removed CI/CD labels Dec 1, 2022
Copy link
Contributor

@MaxJPRey MaxJPRey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@Revathyvenugopal162 Revathyvenugopal162 merged commit 5838345 into main Dec 1, 2022
@Revathyvenugopal162 Revathyvenugopal162 deleted the ci/migrate-to-pyansys-actions branch December 1, 2022 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Package and maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup pyoptics to use pyansys actions.
3 participants